Skip to content
This repository was archived by the owner on Nov 27, 2017. It is now read-only.

Conversation

@k-takata
Copy link
Member

Clangで --coverage が使えることが確認できました。
ついでに、cpp-coverageにエンコーディング変換機能が付いたようなので、有効にしてみました。

@crazymaster
Copy link
Member

coveralls側で変更があったようで、Source not availableになっています。
ref. lemurheavy/coveralls-public#263

おそらく cd せずにオプションでディレクトリを渡す必要があるのだと思います。

@coveralls
Copy link

Coverage Status

Coverage increased (+2.37%) when pulling 18642c2 on enable-coverage-with-clang into 1985a6a on master.

@mattn
Copy link
Member

mattn commented Jun 26, 2014

意外と高かった

@k-takata
Copy link
Member Author

おそらく cd せずにオプションでディレクトリを渡す必要があるのだと思います。

残念ながら失敗。どうすればいいのかよく分かっていません。

@crazymaster
Copy link
Member

Github APIでソースを取得するようにしたと書いてあるから、
submodule ではダメになったのかも?

@k-takata
Copy link
Member Author

ちなみに、自分のプロジェクトだと cpp-coveralls よりも lcoveralls の方が簡単に設定できたので、lcoveralls も試してみたのですが、一部のファイルのエンコーディングの問題と、if_perl.c のせいでこちらもうまくいっていません。

Also enable encoding conversion of some source code files.
@coveralls
Copy link

Coverage Status

Coverage increased (+2.68%) when pulling 02814c3 on enable-coverage-with-clang into 949aa54 on master.

@k-takata
Copy link
Member Author

k-takata commented Jul 3, 2014

18642c2 の修正はSource not availableに対する効果がなかったようなので、削除してコミットを整理しました。
Source not availableは別issueにしましょう。

k-takata added a commit that referenced this pull request Jul 3, 2014
@k-takata k-takata merged commit a82b66b into master Jul 3, 2014
@k-takata k-takata deleted the enable-coverage-with-clang branch July 3, 2014 03:21
@k-takata k-takata mentioned this pull request Jul 3, 2014
@k-takata
Copy link
Member Author

k-takata commented Jul 3, 2014

特に反対もありませんでしたので、マージしてしまいました。

@k-takata k-takata mentioned this pull request Jul 3, 2014
@koron
Copy link
Member

koron commented Jul 3, 2014

+100

@crazymaster
Copy link
Member

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants